-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cpython: several fixes #10795
cpython: several fixes #10795
Conversation
This comment has been minimized.
This comment has been minimized.
I detected other pull requests that are modifying cpython/all recipe:
This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there. |
This comment has been minimized.
This comment has been minimized.
@@ -178,7 +183,7 @@ def requirements(self): | |||
self.requires("openssl/1.1.1l") | |||
self.requires("expat/2.4.1") | |||
if self._with_libffi: | |||
self.requires("libffi/3.2.1") | |||
self.requires("libffi/3.4.2") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You might want to make the libffi version dependent of the python version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's working, then it's fine.
But in my experience with cpython + libffi, there is some coupling with libffi versioning and cpython versions.
recipes/cpython/all/patches/3.10.3-0006-restore-support-for-windows-7.patch
Outdated
Show resolved
Hide resolved
recipes/cpython/all/patches/3.10.3-0005-do-not-search-for-system-tkinter.patch
Outdated
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
Well, I see that cpython/3.9.7 Linux clang 11 build fails. Some strange error related to |
Tbh, I also have no clue what is going wrong. Perhaps @jgsogo is able to create a reproducer since he has most(?) knowledge about the internals of c3i. |
@uilianries do you rememeber what was the problem with |
btw, the same problem in #10706 |
clang 11 uses old docker images: |
Clang 10 and Clang 11 are fixed now: conan-io/conan-docker-tools#435 I just relaunched this PR now. |
Well, cpython/3.8.12 works in my machine ™ I'm running the latest 'conanio/clang11:1.47.0' image available in hub.docker Here is my log: cpython_3.8.12.log To execute my steps:
The result is in the file above. Now, why is it failing ... Does Ci didn't update the consumed docker image yet? |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Good news! Linux images are now running, we needed some interval to sync all images I guess. I just re-executed again and cpython/3.8.12 passed on Linux. However, now Mac is the new failure. |
@uilianries can you check local builds on mac? especially, inspecting the |
Hooks produced the following warnings for commit 0d5fc22cpython/3.9.7
|
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit bed1631cpython/3.9.7
|
Hook fails with |
I am not that good with components. maybe @uilianries @danimtb @jgsogo can suggest. |
TBH, I don't understand the error, component |
This comment was marked as outdated.
This comment was marked as outdated.
KB-H054 and KB-H071 unblocks conan-io/conan-center-index#10795
KB-H054 and KB-H071 unblocks conan-io/conan-center-index#10795
This should be fixed now that conan-io/hooks#425 is merged. Let's relaunch the CI |
This comment has been minimized.
This comment has been minimized.
All green in build 21 (
|
Hooks produced the following warnings for commit d0b1137cpython/3.10.0
cpython/3.8.12
cpython/3.10.3
cpython/3.9.7
cpython/3.7.12
|
@madebr
|
Perhaps this is the reason the Windows CPython installer installs python in a C drive root folder by default.
This is probably used for the repl interpreter. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This pull request has been automatically closed because it has not had recent activity. Thank you for your contributions. |
Specify library name and version: cpython/*
devenv
is not available (e.g. when using BuildTools)with_curses=False
with_tkinter=False